Flakyなテストを2つ修正した #7997
Merged
Flakyなテストを2つ修正した #7997
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
test/system/users.test.rb:436
出ているエラーエラー
対策
|
test/system/require_login/users_test.rb:8
出ているエラーエラー
test (37018) - fjordllc/bootcamp
調査の流れ
対策
CIの結果他テストファイル含む 修正前：1/10 → 修正後：0/25 |
motohiro-mm
commented
Aug 9, 2024
| @@ -2,31 +2,31 @@ | |||
| # ActiveStorage::FixtureSet.blob を使うように変更する | |||
|
|
|||
There was a problem hiding this comment.
こちらを参考に、各フィクスチャの保存先を設定しています。
ちなみに、BootcamoアプリはRails6.1であり、RailsガイドにあるようなActiveStorage::FixtureSet.blobという機能はRails7からの機能なのでありませんが、こちらのPRのtest/system/test_helper.rbによってActiveStorage::Blob.fixtureとすることで同等の機能が使用できるようになっています。
| @@ -15,6 +15,11 @@ google: | |||
| bucket: <%= ENV["GCS_BUCKET"] %> | |||
| public: true | |||
|
|
|||
There was a problem hiding this comment.
下記リンクを参考にfixtures用の保存先を作成しています。
Active Storage の概要 - Railsガイド
| @@ -32,6 +32,22 @@ class ActiveSupport::TestCase | |||
| teardown do | |||
| ActiveStorage::Current.host = nil | |||
| end | |||
|
|
|||
There was a problem hiding this comment.
こちらを参考にして、並列のプロセスごとに、保存先にそれぞれ番号がふられるように設定しています。
Railsガイドにある書き方はRails7から入った機能です。
Rails6.1で同様のことをするためinstance_variable_setを使用して修正しています。
そしてBootcampアプリがRails7になったときのためにコメントを記載しています。
参考：Parallelizing setup for ActiveStorage doesnt expose setter · Issue #43616 · rails/rails
（ちなみにRails6.1のRailsガイドには並列実行時のテスト用保存先の別名の付け方の記載はありません）
| ActiveStorage::Blob.service.instance_variable_set(:@root, "#{ActiveStorage::Blob.service.root}/storage-#{i}") | ||
| ActiveStorage::Blob.services.fetch(:test_fixtures).instance_variable_set(:@root, "#{ActiveStorage::Blob.services.fetch(:test_fixtures).root}/fixtures-#{i}") | ||
| end | ||
|
|
There was a problem hiding this comment.
アップロード用保存先フォルダに関しては、Railsガイドにはafter_teardownを使用し各テストごとに削除するよう記載してありますが、そのように記載しCIを実行するとActiveStorage::FileNotFoundError: ActiveStorage::FileNotFoundErrorが発生する回数が増えました。
なぜエラーが増えるのか調査しましたが、理由はわかりませんでした。
そのため、テストが全て終了した後にデータをクリーンアップするように設定しています。
motohiro-mm
force-pushed
the
chore/fix-flaky-tests
branch
from
c349fc2
to
1342612
Compare
|
おつかれさまです！ |
|
@motohiro-mm |
|
急ぎではありません！ |
|
@motohiro-mm |
|
ご確認いただきありがとうございます！ WARNINGの解決策はわかっていないのですが、とりあえず確認していただきたいことを以下に書いておきます↓
もし的外れだったらすみません🙇♀️ |
kurumadaisuke
approved these changes
Aug 16, 2024
|
WARNINGが消えたようでよかったです！🙌 |
|
おつかれさまです！ |
17 tasks
Closed
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Issue
概要
CircleCIの test insights の中のFlakyなテストから、以下のテストについて修正・追加の実装を行いました。
１：
test/system/users.test.rb:436２：
test/system/require_login/users_test.rb:8各テストにおける修正・追加の実装の説明は以下コメントで記載します。
変更確認方法
chore/fix-flaky-testsをローカルに取り込むtest/system/users.test.rb:436test/system/require_login/users_test.rb:8test/system/users.test.rb:436test/system/require_login/users_test.rb:8